Skip to content

updated libprotobuf script with ubi10.2 and python3.14 support - #8697

Open
tejasBadjateIBM wants to merge 9 commits into
ppc64le:masterfrom
tejasBadjateIBM:libprotobuf_ubi10.2
Open

updated libprotobuf script with ubi10.2 and python3.14 support#8697
tejasBadjateIBM wants to merge 9 commits into
ppc64le:masterfrom
tejasBadjateIBM:libprotobuf_ubi10.2

Conversation

@tejasBadjateIBM

Copy link
Copy Markdown
Contributor

Checklist

  • Have you checked and followed all the points mention in the CONTRIBUTING.MD
  • Have you validated script on UBI 9 container
  • Did you run the script(s) on fresh container with set -e option enabled and observe success ?
  • Did you have Legal approvals for patch files ?

Comment thread l/libprotobuf/libprotobuf_ubi_10.2.sh Outdated
wget https://raw.githubusercontent.com/ppc64le/build-scripts/refs/heads/master/l/libprotobuf/pyproject.toml
sed -i "s/{PACKAGE_VERSION}/$PACKAGE_VERSION/g" pyproject.toml

python3.12 -m pip wheel -w $WORK_DIR -vv --no-build-isolation --no-deps .

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update it to use the correct Python version, i.e. Python 3.14

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

updated the script can you please review again

@prathamesh-ibm prathamesh-ibm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@rashmi-ibm rashmi-ibm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants